Conversation
suite à un nouvel export de sébastien
| \"energie_ratio_autreChaleurRecuperee\", | ||
| \"energie_ratio_biogaz\" | ||
| ) as \"energie_max_ratio\" | ||
| * |
There was a problem hiding this comment.
A un moment donné, on avait besoin de l'énergie majoritaire, mais ce n'est plus le cas, donc on simplifie la requête.
| @@ -15,7 +15,7 @@ | |||
| 3. Mise à jour des données sur les réseaux depuis Airtable | |||
| - Si la table des réseaux a été mise à jour lors de l'étape précédente : `yarn cli update-networks network` | |||
| - Sinon | |||
| - `yarn cli download-network network` | |||
| - `yarn cli download-update-network network` | |||
There was a problem hiding this comment.
J'ai changé un peu le processus, et je voulais que download-network (= synchro de airtable vers postgres) n'ait pas d'effets de bord et ne modifie pas airtable. J'ai donc changé la commande référencée ici.
Ca devrait bouger aussi un peu à l'avenir.
| .argument('[zoomMin]', 'Minimum zoom', parseInt, 0) | ||
| .argument('[zoomMax]', 'Maximum zoom', parseInt, 17) | ||
| .argument('[zoomMin]', 'Minimum zoom', (v) => parseInt(v), 0) | ||
| .argument('[zoomMax]', 'Maximum zoom', (v) => parseInt(v), 17) |
There was a problem hiding this comment.
En fait il n'y a pas que v qui est passé en argument et ça passait directement en 2e argument de parseInt comme base. Et on veut pas ça...
| } catch (err) { | ||
| console.error('err', err); | ||
| process.exit(2); | ||
| } |
There was a problem hiding this comment.
Je pensais qu'il y avait un catch global pour chaque action mais que nenni. Il faudrait que ça soit global pour pas s'embêter avec des try catch.
| @@ -11,7 +11,7 @@ table=$2 | |||
| options=$3 | |||
| if [[ $env != "dev" && $env != "prod" ]]; then | |||
| usage | |||
| exit 1+ | |||
| exit 1 | |||
| @@ -48,7 +48,7 @@ const conversionConfigReseauxDeChaleur = { | |||
| 'Dev_reseau%': TypeNumber, | |||
| 'Rend%': TypeNumber, | |||
| reseaux_techniques: TypeBool, | |||
| departement: TypeNumber, | |||
| departement: TypeString, | |||
There was a problem hiding this comment.
La colonne departement contenait jusque là des codes départements / codes postaux. Et ça contient maintenant des labels... Du coup j'ai changé le type, et il faudra dans le futur compléter cette information automatiquement. C'est utilisé que côté airtable.
| @@ -525,7 +525,7 @@ export function buildMapLayers(config: MapConfiguration): MapSourceLayersSpecifi | |||
| source: { | |||
| type: 'vector', | |||
| tiles: [`${location.origin}/api/map/zoneDP/{z}/{x}/{y}`], | |||
| maxzoom: tileSourcesMaxZoom, | |||
| maxzoom: 14, | |||
There was a problem hiding this comment.
Changé car pas besoin d'avoir des cases plus petites. (qui prennent aussi du temps à être générées)
| ST_Transform('SRID=4326;POINT(${lon} ${lat})'::geometry, 2154), | ||
| geom | ||
| ) | ||
| `) |
There was a problem hiding this comment.
Bon j'ai pas mal galéré aussi avec ça. Globalement le cast <boolean> est important pour kysely. Mais après j'ai dû utiliser sql.raw car je m'en sortais pas avec l'erreur bind message supplies 2 parameters, but prepared statement "" requires 0
Cette PR introduit quelques scripts utilisés pour mettre à jour les tracés et les données. Pas mal de synchro postgres - airtable.
Il y aura encore peut-être des changements pour affiner le processus avec les futures mises à jour côté Sébastien.
Le but était de vraiment maitriser les changements faits et noter toutes les modifications. Finalement c'était un peu un bourbier et il y a des choses que j'aurais pu simplifier.
Pour déployer en prod (à faire à peu près pendant le déploiement du build...) :
+ bonus avec la diminution de la hauteur du logo, vu avec Antoine et Florence